Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add SafePal Wallet #56

Merged
merged 3 commits into from
Jun 28, 2024
Merged

Add SafePal Wallet #56

merged 3 commits into from
Jun 28, 2024

Conversation

SafePalWallet
Copy link
Contributor

@SafePalWallet SafePalWallet commented Jun 16, 2024

Add SafePal wallet info

Add <WALLET_NAME> wallet

Supports

  • JS bridge as a browser extention for Google Chrome, ..., browsers.
  • JS bridge for the in-wallet browser for iOS and Android.
  • JS bridge for in-wallet browser for Windows, macOS, ... .
  • HTTP bridge as a mobile wallet app for iOS and Android.
  • HTTP bridge as a desctop wallet app for Windows, macOS, ... .

Supported features

Tests

a demo dapp with integrated wallet

Integrator contacts

  • telegram
  • email
  • discord
  • ...

Add SafePal wallet info
@SafePalWallet
Copy link
Contributor Author

Please test with SafePal Wallet App V4.7.1

@LetikGit
Copy link
Member

Please test with SafePal Wallet App V4.7.1

The last available version is 4.7.0

@SafePalWallet
Copy link
Contributor Author

Please test with SafePal Wallet App V4.7.1

The last available version is 4.7.0

Now the last available version is 4.7.1, please test, thanks

@LetikGit
Copy link
Member

Please test with SafePal Wallet App V4.7.1

The last available version is 4.7.0

Now the last available version is 4.7.1, please test, thanks

The problem still exists. You can check it on dns.ton.org or on sandbox (https://ton-connect.github.io/demo-dapp-with-react-ui/) using the last version for Android (4.7.1)
I tested with the app installed from the APK file, not from Google Play, if it is important.

@SafePalWallet
Copy link
Contributor Author

SafePalWallet commented Jun 20, 2024

Please test with SafePal Wallet App V4.7.1

The last available version is 4.7.0

Now the last available version is 4.7.1, please test, thanks

The problem still exists. You can check it on dns.ton.org or on sandbox (https://ton-connect.github.io/demo-dapp-with-react-ui/) using the last version for Android (4.7.1) I tested with the app installed from the APK file, not from Google Play, if it is important.

I have added deepLink field, "deepLink": "safepal-tc://", the dapp https://ton-connect.github.io/demo-dapp-with-react-ui is updated walletlist-v2.json? Can you tell me the details of the problem?

@LetikGit
Copy link
Member

I have added deepLink field, "deepLink": "safepal-tc://", the dapp https://ton-connect.github.io/demo-dapp-with-react-ui is updated walletlist-v2.json? Can you tell me the details of the problem?

It works well with iOS, but with Android exist some problem.

Step-by-step

  1. I click "Connect wallet"
  2. Choose "SafePal" from the list
  3. It redirects me to link.safepal.io with some GET-params
  4. Then it redirects me to safepal.com/download also with some GET-params and shows me links for SafePal App on different platforms

Screencast:

rec-safepal.mp4

remove universal_url
@SafePalWallet
Copy link
Contributor Author

SafePalWallet commented Jun 24, 2024

I have added deepLink field, "deepLink": "safepal-tc://", the dapp https://ton-connect.github.io/demo-dapp-with-react-ui is updated walletlist-v2.json? Can you tell me the details of the problem?

It works well with iOS, but with Android exist some problem.

Step-by-step

  1. I click "Connect wallet"
  2. Choose "SafePal" from the list
  3. It redirects me to link.safepal.io with some GET-params
  4. Then it redirects me to safepal.com/download also with some GET-params and shows me links for SafePal App on different platforms

Screencast:

rec-safepal.mp4

Now the latest version V4.7.2 is available, please use V4.7.2 version to test. Thanks!

@tolya-yanot tolya-yanot merged commit e32e580 into ton-blockchain:main Jun 28, 2024
1 check passed
@crazyministr
Copy link

@thekiba thekiba mentioned this pull request Sep 9, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants